unit tests for Burger (develop1) #1154
Conversation
.idea/misc.xml
Outdated
There was a problem hiding this comment.
⛔️Нужно исправить. Папку .idea не нужно было загружать в репозиторий. Эта папка должна быть добавлена в .gitignore.
|
|
||
| burger.setBuns(bun); | ||
|
|
||
| assertEquals("black bun", burger.bun.getName()); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для юнит-тестов придерживаемся подхода: один тест, значит одна проверка. Если очень хочется несколько проверок -- тогда используем softAssertions. Поправь, пожалуйста, во всем коде
| } | ||
|
|
||
| @Test | ||
| public void setBuns_shouldAssignBun() { |
There was a problem hiding this comment.
⛔️Нужно исправить. Неверный нейминг. Не используем спецсимволы в названии методов
| public void removeIngredient_shouldRemoveFromList() { | ||
| Burger burger = new Burger(); | ||
| burger.setBuns(new Bun("default", 50.0f)); | ||
| Ingredient ingredient = new Ingredient(IngredientType.FILLING, "cheese", 40.0f); |
There was a problem hiding this comment.
⛔️Нужно исправить. Для всех зависимостей нужно использовать моки
|
|
||
| String receipt = burger.getReceipt(); | ||
|
|
||
| assertTrue(receipt.contains("(==== black bun ====)")); |
There was a problem hiding this comment.
⛔️Нужно исправить. Строку рецепта нужно проверять целиком, чтобы не пропустить ошибки форматирования
| String expectedPriceLine = String.format("Price: %f", burger.getPrice()); | ||
| assertTrue(receipt.contains(expectedPriceLine)); | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
⛔️Нужно исправить. Нужно написать параметризированные тесты.
| private Bun bunMock; | ||
|
|
||
| @Mock | ||
| private Ingredient ingredientMock1; |
There was a problem hiding this comment.
⛔️Нужно исправить. При нейминге не рекомендуется использовать числа (Field2), их еще называют magicNumbers. Очень тяжело поддерживать код с magicNumbers.
| Ingredient ingredient = new Ingredient(type, name, price); | ||
| assertEquals(price, ingredient.getPrice(), 0.0001f); | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
⛔️Нужно исправить. Нужно приложить отчет о тестировании
2 participants
100% coverage for Burger — диплом часть 1